Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[rc] Add variant for the latest AWS SDK on the libevent-2.1.10 spec. #249

Merged
merged 13 commits into from
Jul 12, 2024
Merged

[rc] Add variant for the latest AWS SDK on the libevent-2.1.10 spec. #249

merged 13 commits into from
Jul 12, 2024

Conversation

jjerphan
Copy link
Member

@jjerphan jjerphan commented Jul 10, 2024 •
edited
Loading

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

See added comment in recipe/conda_build_config.yaml

jjerphan added 2 commits July 10, 2024 13:47
@jjerphan
Add variant for the latest AWS SDK on the libevent-2.1.10 spec.
a03c8c2 
Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan
Bump build number
e746f03 
Necessary since a new version of the C standard library is used.

Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan
Copy link
Member Author

@conda-forge-admin, please rerender

@conda-forge-webservices
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jjerphan jjerphan marked this pull request as ready for review July 10, 2024 12:20
@jjerphan
Copy link
Member Author

Failures are due to crash in a test suite.

Likely related to new minor release of libraries in the AWS stack.

Unlikely related to the update of the C standard library (from 2.16 to 2.17).

Diff of dependencies between the previous build and this one for osx_64_aws_sdk_cpp_azure_core_cpp_libevent_numpy1.22python3.9.____cpython:

1,2c1,2
< aws-c-auth:               0.7.22-h2f80047_6           conda-forge
< aws-c-cal:                0.6.15-hd73d8db_1           conda-forge
---
> aws-c-auth:               0.7.22-he52cbe7_7           conda-forge
> aws-c-cal:                0.7.0-hd73d8db_0            conda-forge
6,7c6,7
< aws-c-http:               0.8.2-hc7634fe_3            conda-forge
< aws-c-io:                 0.14.9-h2089a17_3           conda-forge
---
> aws-c-http:               0.8.2-hf609411_4            conda-forge
> aws-c-io:                 0.14.9-h9f49e27_5           conda-forge
9c9
< aws-c-s3:                 0.5.10-h7347a4b_4           conda-forge
---
> aws-c-s3:                 0.6.0-h8fdb93b_0            conda-forge
12,13c12,13
< aws-crt-cpp:              0.26.12-hc167df4_1          conda-forge
< aws-sdk-cpp:              1.11.329-h4fac305_6         conda-forge
---
> aws-crt-cpp:              0.27.2-h7746516_0           conda-forge
> aws-sdk-cpp:              1.11.329-hce9e41e_8         conda-forge
22c22
< ca-certificates:          2024.6.2-h8857fd0_0         conda-forge
---
> ca-certificates:          2024.7.4-h8857fd0_0         conda-forge
42c42
< libcxx:                   17.0.6-h8d2f0da_2           conda-forge
---
> libcxx:                   18.1.8-hef8daea_0           conda-forge
56c56
< libopenblas:              0.3.27-openmp_hfef2a42_0    conda-forge
---
> libopenblas:              0.3.27-openmp_h8869122_1    conda-forge
83,84c83,84
< setuptools:               70.1.1-pyhd8ed1ab_0         conda-forge
< snappy:                   1.2.0-h6dc393e_1            conda-forge
---
> setuptools:               70.2.0-pyhd8ed1ab_0         conda-forge
> snappy:                   1.2.1-he1e6707_0            conda-forge

@jjerphan jjerphan marked this pull request as draft July 10, 2024 15:49
@jjerphan
Copy link
Member Author

@conda-forge-admin, please rerender

@github-actions GitHub Actions
Copy link
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you but ran into some issues. Please check the output logs of the latest webservices GitHub actions workflow run for errors. You can also ping conda-forge/core for further assistance or you can try rerendeing locally.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/arcticdb-feedstock/actions/runs/9877403193.

@jjerphan jjerphan force-pushed the rc-new-pin-for-latest-aws-sdk branch from dd54d9a to 8d79eb4 Compare July 10, 2024 15:56
@jjerphan
Copy link
Member Author

@conda-forge-admin, please rerender

@github-actions GitHub Actions
Copy link
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/arcticdb-feedstock/actions/runs/9877504039.

@jjerphan jjerphan force-pushed the rc-new-pin-for-latest-aws-sdk branch from 8d79eb4 to 475e3e7 Compare July 11, 2024 07:32
@jjerphan
Copy link
Member Author

@conda-forge-admin, please rerender

@jjerphan
DEBUG
5f9f42b 
Signed-off-by: Julien Jerphanion <[email protected]>
@github-actions GitHub Actions
Copy link
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/arcticdb-feedstock/actions/runs/9887375365.

@jjerphan jjerphan force-pushed the rc-new-pin-for-latest-aws-sdk branch from 475e3e7 to 5f9f42b Compare July 11, 2024 07:36
jjerphan added 2 commits July 11, 2024 14:39
@jjerphan
DEBUG
0a22550 
Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan
DEBUG
6a5587c 
Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan
Copy link
Member Author

@conda-forge-admin, please rerender

conda-forge-webservices[bot] and others added 3 commits July 11, 2024 15:10
MNT: Re-rendered with conda-build 24.5.1, conda-smithy 3.36.2, and co…
ff56040 
…nda-forge-pinning 2024.07.11.08.43.30
@jjerphan
Revert "DEBUG"
bdb6ae1 
This reverts commit 6a5587c.
@jjerphan
DEBUG Use libcxx <18 at test time
465a374 
Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan
Copy link
Member Author

@conda-forge-admin, please rerender

@jjerphan jjerphan mentioned this pull request Jul 12, 2024
Closed
@jjerphan jjerphan marked this pull request as ready for review July 12, 2024 09:53
jjerphan added 2 commits July 12, 2024 12:01
@jjerphan
Only omit the problematic build
d288994 
Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan
Use previous pin
348aede 
The MatchSpec `libcxx !=18.1.8 hef8daea_0` is supported
but is unusable apparently.

Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan jjerphan force-pushed the rc-new-pin-for-latest-aws-sdk branch from 1af4882 to 348aede Compare July 12, 2024 13:26
@jjerphan jjerphan merged commit fdd6b7d into conda-forge:rc Jul 12, 2024
24 checks passed
@jjerphan jjerphan deleted the rc-new-pin-for-latest-aws-sdk branch July 12, 2024 15:56
@jjerphan jjerphan mentioned this pull request Jul 30, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@h-vetinari h-vetinari h-vetinari left review comments

@DrNickClarke DrNickClarke DrNickClarke approved these changes

@DerThorsten DerThorsten Awaiting requested review from DerThorsten DerThorsten is a code owner

@G-D-Petrov G-D-Petrov Awaiting requested review from G-D-Petrov G-D-Petrov is a code owner

@Hind-M Hind-M Awaiting requested review from Hind-M Hind-M is a code owner

@JohanMabille JohanMabille Awaiting requested review from JohanMabille JohanMabille is a code owner

@Klaim Klaim Awaiting requested review from Klaim Klaim is a code owner

@alexowens90 alexowens90 Awaiting requested review from alexowens90 alexowens90 is a code owner

@jamesmunro jamesmunro Awaiting requested review from jamesmunro jamesmunro is a code owner

@joe-iddon joe-iddon Awaiting requested review from joe-iddon joe-iddon is a code owner

@mehertz mehertz Awaiting requested review from mehertz mehertz is a code owner

@phoebusm phoebusm Awaiting requested review from phoebusm phoebusm is a code owner

@poodlewars poodlewars Awaiting requested review from poodlewars poodlewars is a code owner

@qc00 qc00 Awaiting requested review from qc00 qc00 is a code owner

@vasil-pashov vasil-pashov Awaiting requested review from vasil-pashov vasil-pashov is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jjerphan @DrNickClarke @h-vetinari